Skip to content

fix(processCss/processNode): don't support AlphaImageLoader(src='/xxx') #293

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

netwjx
Copy link

@netwjx netwjx commented Jun 28, 2016

No description provided.

@codecov-io
Copy link

codecov-io commented Jun 28, 2016

Current coverage is 96.87%

No coverage report found for master at b6acfec.

Powered by Codecov. Last updated by b6acfec...38a5ecb

@sokra
Copy link
Member

sokra commented Jul 26, 2016

Please add test cases for this behavior.

@michael-ciniawsky
Copy link
Member

@netwjx What exactly breaks 'AlphaImageLoader', can you elaborate on this case in a more generic way ? 😛 If this should be a fix for a single uncommon loader alone, I have to reject this PR

@michael-ciniawsky
Copy link
Member

@netwjx I'm interpreting the 👍 as 'yep, this PR is a specific change' and close the PR for now, if my assumption should be wrong, feel free to reopen 😛

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants